Changeset View
Changeset View
Standalone View
Standalone View
clang/include/clang/Basic/DiagnosticSemaKinds.td
- This file is larger than 256 KB, so syntax highlighting is disabled by default.
Show First 20 Lines • Show All 2,845 Lines • ▼ Show 20 Lines | def err_alignment_too_small : Error< | ||||
"requested alignment must be %0 or greater">; | "requested alignment must be %0 or greater">; | ||||
def err_alignment_too_big : Error< | def err_alignment_too_big : Error< | ||||
"requested alignment must be %0 or smaller">; | "requested alignment must be %0 or smaller">; | ||||
def err_alignment_not_power_of_two : Error< | def err_alignment_not_power_of_two : Error< | ||||
"requested alignment is not a power of 2">; | "requested alignment is not a power of 2">; | ||||
def err_alignment_dependent_typedef_name : Error< | def err_alignment_dependent_typedef_name : Error< | ||||
"requested alignment is dependent but declaration is not dependent">; | "requested alignment is dependent but declaration is not dependent">; | ||||
def err_attribute_aligned_too_great : Error< | def err_attribute_aligned_too_great : Error< | ||||
hfinkel: Should this be in the IgnoredAttributes group? The builtin is not an attribute.
Also, I'd… | |||||
"requested alignment must be %0 bytes or smaller">; | "requested alignment must be %0 bytes or smaller">; | ||||
def warn_assume_aligned_too_great | |||||
: Warning<"requested alignment must be %0 bytes or smaller; assumption " | |||||
"ignored">, | |||||
InGroup<DiagGroup<"builtin-assume-aligned-alignment">>; | |||||
rsmithUnsubmitted Ignoring the assumption in this case seems unnecessarily user-hostile (and would require hard-coding an arbitrary LLVM limit in the source code to work around). Couldn't we just assume the highest alignment that we do support instead? rsmith: Ignoring the assumption in this case seems unnecessarily user-hostile (and would require hard… | |||||
erichkeaneAuthorUnsubmitted This more closely fit GCCs behavior (of ignoring any invalid value, though their top limit is int64-max). We can definitely just assume highest alignment, I'll do that in my re-land. erichkeane: This more closely fit GCCs behavior (of ignoring any invalid value, though their top limit is… | |||||
def warn_redeclaration_without_attribute_prev_attribute_ignored : Warning< | def warn_redeclaration_without_attribute_prev_attribute_ignored : Warning< | ||||
"%q0 redeclared without %1 attribute: previous %1 ignored">, | "%q0 redeclared without %1 attribute: previous %1 ignored">, | ||||
InGroup<MicrosoftInconsistentDllImport>; | InGroup<MicrosoftInconsistentDllImport>; | ||||
def warn_redeclaration_without_import_attribute : Warning< | def warn_redeclaration_without_import_attribute : Warning< | ||||
"%q0 redeclared without 'dllimport' attribute: 'dllexport' attribute added">, | "%q0 redeclared without 'dllimport' attribute: 'dllexport' attribute added">, | ||||
InGroup<MicrosoftInconsistentDllImport>; | InGroup<MicrosoftInconsistentDllImport>; | ||||
def warn_dllimport_dropped_from_inline_function : Warning< | def warn_dllimport_dropped_from_inline_function : Warning< | ||||
"%q0 redeclared inline; %1 attribute ignored">, | "%q0 redeclared inline; %1 attribute ignored">, | ||||
▲ Show 20 Lines • Show All 7,123 Lines • Show Last 20 Lines |
Should this be in the IgnoredAttributes group? The builtin is not an attribute.
Also, I'd rather that the wording were similar to that used by err_attribute_aligned_too_great and other errors, and use 268435456 (which is what we get for MaxValidAlignment based on LLVM restrictions) instead of '1 << 29'.