Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LLVM 13 #112

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

LLVM 13 #112

wants to merge 5 commits into from

Conversation

jacky860226
Copy link
Contributor

@jacky860226 jacky860226 commented Aug 20, 2021

@jacky860226 jacky860226 requested a review from hydai August 20, 2021 13:45
@LFsWang LFsWang force-pushed the yul_metadata branch 3 times, most recently from b131520 to c4413c3 Compare November 23, 2021 18:05
@codecov
Copy link

codecov bot commented Nov 23, 2021

Codecov Report

Merging #112 (b2bc554) into master (dd539e0) will increase coverage by 0.01%.
The diff coverage is 75.75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #112      +/-   ##
==========================================
+ Coverage   66.47%   66.48%   +0.01%     
==========================================
  Files          98       98              
  Lines       13679    13743      +64     
  Branches     2041     2047       +6     
==========================================
+ Hits         9093     9137      +44     
- Misses       3567     3585      +18     
- Partials     1019     1021       +2     
Impacted Files Coverage Δ
include/soll/AST/ASTContext.h 100.00% <ø> (ø)
include/soll/Basic/FileManager.h 100.00% <ø> (ø)
include/soll/Basic/IdentifierTable.h 96.15% <ø> (ø)
lib/CodeGen/CodeGenModule.h 96.55% <ø> (ø)
lib/Frontend/ASTConsumers/ASTPrinter.cpp 11.22% <ø> (ø)
lib/Parse/ParseYul.cpp 88.88% <ø> (ø)
lib/Sema/SemaExprAsm.cpp 91.12% <28.57%> (-0.78%) ⬇️
lib/CodeGen/BackendUtil.cpp 53.50% <54.54%> (ø)
lib/CodeGen/CodeGenModule.cpp 64.70% <66.19%> (-0.02%) ⬇️
lib/CodeGen/CodeGenAction.cpp 74.64% <66.66%> (+0.70%) ⬆️
... and 10 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd539e0...b2bc554. Read the comment docs.

@LFsWang LFsWang changed the title LLVM 12 [WIP] LLVM 13 Nov 23, 2021
@lgtm-com
Copy link

lgtm-com bot commented Nov 23, 2021

This pull request introduces 1 alert when merging c4413c3 into ab2b60d - view on LGTM.com

new alerts:

  • 1 for FIXME comment

@lgtm-com
Copy link

lgtm-com bot commented Nov 23, 2021

This pull request introduces 1 alert when merging 833e078 into ab2b60d - view on LGTM.com

new alerts:

  • 1 for FIXME comment

@LFsWang LFsWang changed the title [WIP] LLVM 13 LLVM 13 Nov 24, 2021
@lgtm-com
Copy link

lgtm-com bot commented Nov 24, 2021

This pull request introduces 1 alert when merging b1ca5cf into ab2b60d - view on LGTM.com

new alerts:

  • 1 for FIXME comment

@lgtm-com
Copy link

lgtm-com bot commented Nov 30, 2021

This pull request introduces 1 alert when merging 1f1b6b3 into ab2b60d - view on LGTM.com

new alerts:

  • 1 for FIXME comment

@lgtm-com
Copy link

lgtm-com bot commented Dec 1, 2021

This pull request introduces 1 alert when merging 16a788f into ab2b60d - view on LGTM.com

new alerts:

  • 1 for FIXME comment

@LFsWang LFsWang force-pushed the yul_metadata branch 2 times, most recently from 82fe33f to 1be0ff1 Compare December 1, 2021 16:23
@lgtm-com
Copy link

lgtm-com bot commented Dec 1, 2021

This pull request introduces 1 alert when merging 1be0ff1 into ab2b60d - view on LGTM.com

new alerts:

  • 1 for FIXME comment

@lgtm-com
Copy link

lgtm-com bot commented Dec 2, 2021

This pull request introduces 1 alert when merging a31d805 into ab2b60d - view on LGTM.com

new alerts:

  • 1 for FIXME comment

@@ -29,8 +29,7 @@ class Decl {
friend class ASTReader;

protected:
Decl(SourceRange L,
llvm::StringRef Name = llvm::StringRef::withNullAsEmpty(nullptr),
Decl(SourceRange L, llvm::StringRef Name = llvm::StringRef(""),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Decl(SourceRange L, llvm::StringRef Name = {},

@@ -49,7 +49,8 @@ llvm::Value *AbiEmitter::emitEncodeTuple(
std::tie(Value, IsStateVariable) = Values[I];
if (Value->getType()->isDynamic()) {
DynamicPos.emplace_back(Int8Ptr, I);
Int8Ptr = Builder.CreateInBoundsGEP(Int8Ptr, {Builder.getIntN(32, 32)});
Int8Ptr = Builder.CreateInBoundsGEP(Builder.getInt8Ty(), Int8Ptr,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CGM.Int8Ty

@@ -96,7 +97,7 @@ AbiEmitter::getDecode(llvm::Value *Int8Ptr, const Type *Ty) {
Builder.getIntN(32, 32)),
Builder.getIntN(32, 32));
llvm::Value *NextInt8Ptr =
Builder.CreateInBoundsGEP(Int8Ptr, {EncodeLength});
Builder.CreateInBoundsGEP(Builder.getInt8Ty(), Int8Ptr, {EncodeLength});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CGM.Int8Ty

Builder.CreateInBoundsGEP(Int8Ptr, {Builder.getInt32(32)});
llvm::Value *Val = Builder.CreateLoad(ValueTy, ValPtr);
llvm::Value *NextInt8Ptr = Builder.CreateInBoundsGEP(
Builder.getInt8Ty(), Int8Ptr, {Builder.getInt32(32)});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CGM.Int8Ty

Val = CGM.getEndianlessValue(Val);
Val = Builder.CreateZExtOrTrunc(Val, ValueTy);
llvm::Value *NextInt8Ptr = Builder.CreateInBoundsGEP(
Int8Ptr, {Builder.getInt32(ValueTy->getIntegerBitWidth() / 8)});
Builder.getInt8Ty(), Int8Ptr,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CGM.Int8Ty

llvm::Value *Ptr =
Builder.CreateInBoundsGEP(CGM.getHeapBase(), {Pos}, "heap.cptr");
llvm::Value *Ptr = Builder.CreateInBoundsGEP(
CGM.getHeapBase()->getType()->getPointerElementType(), CGM.getHeapBase(),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CGM.Int8Ty

@@ -1015,13 +1025,15 @@ void CodeGenFunction::emitAsmSetImmutable(const CallExpr *CE) {
llvm::Value *Offset = emitExpr(Arguments[0])->load(Builder, CGM);
llvm::Value *TableOffset = Builder.getInt(Map.lookup(StringRefName));
llvm::Value *ImmutableCPtr = Builder.CreateInBoundsGEP(
CGM.getImmutableBase()->getType()->getPointerElementType(),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CGM.Int256Ty

llvm::Value *HeapCPtr =
Builder.CreateInBoundsGEP(CGM.getHeapBase(), {Offset}, "heap.cptr");
llvm::Value *HeapCPtr = Builder.CreateInBoundsGEP(
CGM.getHeapBase()->getType()->getPointerElementType(),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CGM.Int8Ty

llvm::Value *HeapCPtr =
Builder.CreateInBoundsGEP(CGM.getHeapBase(), {Offset}, "heap.cptr");
llvm::Value *HeapCPtr = Builder.CreateInBoundsGEP(
CGM.getHeapBase()->getType()->getPointerElementType(),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CGM.Int8Ty

@@ -1044,13 +1056,17 @@ llvm::Value *CodeGenFunction::emitAsmLoadImmutable(const CallExpr *CE) {
if (Map.find(StringRefName) != Map.end()) {
llvm::Value *TableOffset = Builder.getInt(Map.lookup(StringRefName));
llvm::Value *ImmutableCPtr = Builder.CreateInBoundsGEP(
CGM.getImmutableBase()->getType()->getPointerElementType(),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CGM.Int256Ty

@lgtm-com
Copy link

lgtm-com bot commented Dec 12, 2021

This pull request introduces 1 alert when merging f0c50ea into ab2b60d - view on LGTM.com

new alerts:

  • 1 for FIXME comment

@hydai hydai requested a review from ibmibmibm December 16, 2021 06:03
@lgtm-com
Copy link

lgtm-com bot commented Dec 23, 2021

This pull request introduces 1 alert when merging b2bc554 into dd539e0 - view on LGTM.com

new alerts:

  • 1 for FIXME comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants