diff options
| author | Philip Reames <listmail@philipreames.com> | 2021-04-14 14:43:53 -0700 |
|---|---|---|
| committer | Philip Reames <listmail@philipreames.com> | 2021-04-14 14:45:24 -0700 |
| commit | 61a85da88235983da565bda0160367461fa0f382 (patch) | |
| tree | e86ecc5f14c325b7e8c52aed5312d7a9bd5fad2a /llvm/tools/llvm-objdump/llvm-objdump.cpp | |
| parent | 61500016a59616a8f5ba5a389bafec3b33845279 (diff) | |
| download | llvm-61a85da88235983da565bda0160367461fa0f382.zip llvm-61a85da88235983da565bda0160367461fa0f382.tar.gz llvm-61a85da88235983da565bda0160367461fa0f382.tar.bz2 | |
[InferAttributes] Materialize all infered attributes for declaration
We have some cases today where attributes can be inferred from another on access, but the result is not explicitly materialized in IR. This change is a step towards changing that.
Why? Two main reasons:
* Human clarity. It's really confusing trying to figure out why a transform is triggering when the IR doesn't appear to have the required attributes.
* This avoids the need to special case declarations in e.g. functionattrs. Since we can assume the attribute is present, we can work directly from attributes (and only attributes) without also needing to query accessors on Function to avoid missing cases due to unannotated (but infered on use) declarations. (This piece will appear must easier to follow once D100226 also lands.)
Differential Revision: https://reviews.llvm.org/D100400
Diffstat (limited to 'llvm/tools/llvm-objdump/llvm-objdump.cpp')
0 files changed, 0 insertions, 0 deletions