diff options
author | Ted Kremenek <kremenek@apple.com> | 2007-12-11 21:27:55 +0000 |
---|---|---|
committer | Ted Kremenek <kremenek@apple.com> | 2007-12-11 21:27:55 +0000 |
commit | 7a9d49fd2bfac00e905b361ba76d26ab5b6c3b09 (patch) | |
tree | 3557aeb775ca6ac99524afd56b1c8840b80d7a94 /Parse/DeclSpec.cpp | |
parent | e66894c3dea9aeab4403e67bf5baabc20da61cb3 (diff) |
Mega-patch: ripped SourceManager out of Diagnostic/DiagnosticClient. Now
SourceManager is passed by reference, allowing the SourceManager to be
associated with a specific translation unit, and not the entire execution
of the driver.
Modified all users of Diagnostics to comply with this new interface.
Integrated SourceManager as a member variable of TargetInfo. TargetInfo will
eventually be associated with a single translation unit (just like
SourceManager).
Made the SourceManager reference in ASTContext private. Provided accessor
getSourceManager() for clients to use instead. Modified clients to comply with
new interface.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@44878 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'Parse/DeclSpec.cpp')
-rw-r--r-- | Parse/DeclSpec.cpp | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/Parse/DeclSpec.cpp b/Parse/DeclSpec.cpp index 19e643d27c..b50c1a7958 100644 --- a/Parse/DeclSpec.cpp +++ b/Parse/DeclSpec.cpp @@ -209,7 +209,8 @@ bool DeclSpec::SetFunctionSpecInline(SourceLocation Loc, const char *&PrevSpec){ /// "_Imaginary" (lacking an FP type). This returns a diagnostic to issue or /// diag::NUM_DIAGNOSTICS if there is no error. After calling this method, /// DeclSpec is guaranteed self-consistent, even if an error occurred. -void DeclSpec::Finish(Diagnostic &D, const LangOptions &Lang) { +void DeclSpec::Finish(Diagnostic &D, SourceManager& SrcMgr, + const LangOptions &Lang) { // Check the type specifier components first. // signed/unsigned are only valid with int/char. @@ -217,7 +218,7 @@ void DeclSpec::Finish(Diagnostic &D, const LangOptions &Lang) { if (TypeSpecType == TST_unspecified) TypeSpecType = TST_int; // unsigned -> unsigned int, signed -> signed int. else if (TypeSpecType != TST_int && TypeSpecType != TST_char) { - Diag(D, TSSLoc, diag::err_invalid_sign_spec, + Diag(D, TSSLoc, SrcMgr, diag::err_invalid_sign_spec, getSpecifierName( (TST)TypeSpecType)); // signed double -> double. TypeSpecSign = TSS_unspecified; @@ -232,7 +233,7 @@ void DeclSpec::Finish(Diagnostic &D, const LangOptions &Lang) { if (TypeSpecType == TST_unspecified) TypeSpecType = TST_int; // short -> short int, long long -> long long int. else if (TypeSpecType != TST_int) { - Diag(D, TSWLoc, + Diag(D, TSWLoc, SrcMgr, TypeSpecWidth == TSW_short ? diag::err_invalid_short_spec : diag::err_invalid_longlong_spec, getSpecifierName( (TST)TypeSpecType)); @@ -243,7 +244,7 @@ void DeclSpec::Finish(Diagnostic &D, const LangOptions &Lang) { if (TypeSpecType == TST_unspecified) TypeSpecType = TST_int; // long -> long int. else if (TypeSpecType != TST_int && TypeSpecType != TST_double) { - Diag(D, TSWLoc, diag::err_invalid_long_spec, + Diag(D, TSWLoc, SrcMgr, diag::err_invalid_long_spec, getSpecifierName( (TST)TypeSpecType)); TypeSpecType = TST_int; } @@ -254,13 +255,13 @@ void DeclSpec::Finish(Diagnostic &D, const LangOptions &Lang) { // disallow their use. Need information about the backend. if (TypeSpecComplex != TSC_unspecified) { if (TypeSpecType == TST_unspecified) { - Diag(D, TSCLoc, diag::ext_plain_complex); + Diag(D, TSCLoc, SrcMgr, diag::ext_plain_complex); TypeSpecType = TST_double; // _Complex -> _Complex double. } else if (TypeSpecType == TST_int || TypeSpecType == TST_char) { // Note that this intentionally doesn't include _Complex _Bool. - Diag(D, TSTLoc, diag::ext_integer_complex); + Diag(D, TSTLoc, SrcMgr, diag::ext_integer_complex); } else if (TypeSpecType != TST_float && TypeSpecType != TST_double) { - Diag(D, TSCLoc, diag::err_invalid_complex_spec, + Diag(D, TSCLoc, SrcMgr, diag::err_invalid_complex_spec, getSpecifierName( (TST)TypeSpecType)); TypeSpecComplex = TSC_unspecified; } @@ -272,7 +273,7 @@ void DeclSpec::Finish(Diagnostic &D, const LangOptions &Lang) { StorageClassSpec = SCS_extern; // '__thread int' -> 'extern __thread int' } else if (StorageClassSpec != SCS_extern && StorageClassSpec != SCS_static) { - Diag(D, getStorageClassSpecLoc(), diag::err_invalid_thread_spec, + Diag(D, getStorageClassSpecLoc(), SrcMgr, diag::err_invalid_thread_spec, getSpecifierName( (SCS)StorageClassSpec)); SCS_thread_specified = false; } |